Skip to content

refactor: remove stale FlowTemplateParser references from PITest config#120

Merged
MaximumTrainer merged 2 commits intomainfrom
copilot/refactor-migrate-flow-template-parser
Apr 20, 2026
Merged

refactor: remove stale FlowTemplateParser references from PITest config#120
MaximumTrainer merged 2 commits intomainfrom
copilot/refactor-migrate-flow-template-parser

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 20, 2026

FlowTemplateParser and FlowTemplateParserTest were deleted in prior commits but the PITest mutation testing configuration in build.gradle.kts still referenced both, leaving the mutation suite broken.

Changes

  • build.gradle.kts PITest config
    • targetClasses: com.factstore.application.FlowTemplateParsercom.factstore.application.template.TemplateParser
    • targetTests: com.factstore.FlowTemplateParserTestcom.factstore.TemplateParserTest + com.factstore.application.TemplateParserTest (the two files that replaced it)

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • invalid-confluence.example.com
    • Triggering command: /usr/lib/jvm/temurin-21-jdk-amd64/bin/java /usr/lib/jvm/temurin-21-jdk-amd64/bin/java -Dorg.gradle.internal.worker.tmpdir=/home/REDACTED/work/OpenFactstore/OpenFactstore/backend/build/tmp/test/work @/home/REDACTED/.gradle/.tmp/gradle-worker-classpath10316935613023970075txt -Xmx512m -Dfile.encoding=UTF-8 -Duser.country -Duser.language=en -Duser.variant -ea worker.org.gradle.process.internal.worker.GradleWorkerMain 'Gradle Test Executor 2' (dns block)
  • invalid-jira-url-that-does-not-exist.example.com
    • Triggering command: /usr/lib/jvm/temurin-21-jdk-amd64/bin/java /usr/lib/jvm/temurin-21-jdk-amd64/bin/java -Dorg.gradle.internal.worker.tmpdir=/home/REDACTED/work/OpenFactstore/OpenFactstore/backend/build/tmp/test/work @/home/REDACTED/.gradle/.tmp/gradle-worker-classpath10316935613023970075txt -Xmx512m -Dfile.encoding=UTF-8 -Duser.country -Duser.language=en -Duser.variant -ea worker.org.gradle.process.internal.worker.GradleWorkerMain 'Gradle Test Executor 2' (dns block)
  • invalid-jira.example.com
    • Triggering command: /usr/lib/jvm/temurin-21-jdk-amd64/bin/java /usr/lib/jvm/temurin-21-jdk-amd64/bin/java -Dorg.gradle.internal.worker.tmpdir=/home/REDACTED/work/OpenFactstore/OpenFactstore/backend/build/tmp/test/work @/home/REDACTED/.gradle/.tmp/gradle-worker-classpath10316935613023970075txt -Xmx512m -Dfile.encoding=UTF-8 -Duser.country -Duser.language=en -Duser.variant -ea worker.org.gradle.process.internal.worker.GradleWorkerMain 'Gradle Test Executor 2' (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

…ences with TemplateParser

Agent-Logs-Url: https://github.com/MaximumTrainer/OpenFactstore/sessions/e96ea874-12a9-47b5-af52-647d7f33486e

Co-authored-by: MaximumTrainer <1376575+MaximumTrainer@users.noreply.github.com>
Copilot AI changed the title [WIP] Refactor: Migrate FlowTemplateParserTest to KosliTemplateParser refactor: remove stale FlowTemplateParser references from PITest config Apr 20, 2026
Copilot AI requested a review from MaximumTrainer April 20, 2026 18:01
@MaximumTrainer MaximumTrainer marked this pull request as ready for review April 20, 2026 18:24
Copilot AI review requested due to automatic review settings April 20, 2026 18:24
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes the backend mutation testing (PITest) configuration by removing stale references to deleted FlowTemplateParser/FlowTemplateParserTest and pointing PITest at the current TemplateParser implementation and its replacement tests.

Changes:

  • Update PITest targetClasses to com.factstore.application.template.TemplateParser.
  • Update PITest targetTests to include the two replacement TemplateParserTest classes.

@MaximumTrainer MaximumTrainer merged commit 72b0ca1 into main Apr 20, 2026
24 checks passed
@MaximumTrainer MaximumTrainer deleted the copilot/refactor-migrate-flow-template-parser branch April 20, 2026 18:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

refactor: migrate FlowTemplateParserTest to KosliTemplateParser and delete deprecated parser

3 participants